Skip to content

Adapt Sql Coordinates#294

Merged
t-ober merged 59 commits intomainfrom
df/#287-sql-geo
Apr 22, 2025
Merged

Adapt Sql Coordinates#294
t-ober merged 59 commits intomainfrom
df/#287-sql-geo

Conversation

@danielfeismann
Copy link
Member

@danielfeismann danielfeismann commented Mar 24, 2025

resolves #287

@danielfeismann
Use WKBElement Geography for Coordinates
c0ef9c3
@danielfeismann
Merge branch 'main' into df(#287-WKBElement-Geography-for-Coordinates
126c341 
# Conflicts:
#	poetry.lock
@danielfeismann
poetry
f1bcf2c
@danielfeismann
rollback get lat and lon of coordinate
b212581
@danielfeismann
make WeatherValue.time noon optional
e36e8f3
@danielfeismann
adapt lon and lat
44cb898
@danielfeismann
adapt coordinate sql model and adapt tests
ec6c396
@danielfeismann
remove unnecessary things, fmt etc.
0f4f0e0
@danielfeismann
docker compose compatibility
10b47d0
@danielfeismann
skip tests if docker is required but not available
956b564
@danielfeismann
adapt ci
76a1268
@danielfeismann
ci
1daedc9
@danielfeismann
start docker before
5b69a4b
@danielfeismann
start docker
233e6ae
@danielfeismann
try docker desktop
7fb9dcd
@danielfeismann
some more ci tests
7b19549
@danielfeismann
even more ci try
1115862
@danielfeismann
even more ci tries
e1eca76
@danielfeismann
even more more ci tries
6c624fb
@danielfeismann
next try
c51c1f8
@danielfeismann
try another approach
0655873
@danielfeismann
Merge branch 'main' into df(#287-WKBElement-Geography-for-Coordinates
da008e8 
# Conflicts:
#	poetry.lock
@danielfeismann
fmt
7ee0955
@danielfeismann
remove pytest marks
ba22087
@danielfeismann
Merge branch 'main' into df(#287-WKBElement-Geography-for-Coordinates
35cd753 
# Conflicts:
#	poetry.lock
@danielfeismann
poetry
3e9bf5b
@danielfeismann
test additional marks
43297ba
@danielfeismann
remove some marks
5df6fe9
@danielfeismann
rollback fixture conftest
5b8c9a3
@danielfeismann
at mark and try check os again
27e4e22
@danielfeismann danielfeismann mentioned this pull request Mar 24, 2025
Closed
@danielfeismann danielfeismann self-assigned this Mar 24, 2025
@danielfeismann danielfeismann added the code quality Improve code quality label Mar 24, 2025
@danielfeismann danielfeismann marked this pull request as ready for review March 24, 2025 15:14
@danielfeismann danielfeismann requested a review from t-ober March 24, 2025 15:14
@danielfeismann danielfeismann marked this pull request as draft March 24, 2025 16:05
@danielfeismann danielfeismann marked this pull request as ready for review March 25, 2025 17:01
t-ober
t-ober approved these changes Apr 22, 2025
View reviewed changes
Copy link
Contributor

@t-ober t-ober left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@t-ober t-ober merged commit 457541e into main Apr 22, 2025
5 checks passed
@danielfeismann danielfeismann deleted the df/#287-sql-geo branch June 30, 2025 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@t-ober t-ober t-ober approved these changes

Assignees

@danielfeismann danielfeismann

Labels

code quality Improve code quality

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Adapt Sql Coordinates

2 participants

@danielfeismann @t-ober